-
-
Notifications
You must be signed in to change notification settings - Fork 417
Comments
Add Duration.isoformat()#522
Conversation
seandstewart
commented
Nov 28, 2020
@sdispater looks like the only failures left are on Python2.7 and unrelated to my changes.
Any thoughts on this feature?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you meant isoformat()
I would really like this feature, but it seems that even the competing PR #314 released almost 2 years prior to this one has no dev comments as well. Hoping one of these two makes it in eventually.
I did notice this PR doesn't drop the unnecessary portions of the string for 0 values, which makes the resulting format quite verbose.
Evan0000000000
commented
Jan 2, 2025
Is there some hold up on this? Lack of interest? Given how Pendulum has a formatter for datetime objects it seems strange there's not at least some nod towards providing one for durations.
Secrus
commented
Jan 3, 2025
Is there some hold up on this? Lack of interest? Given how Pendulum has a formatter for datetime objects it seems strange there's not at least some nod towards providing one for durations.
The commits in this PR are 4+ years old at this point and significantly out of date with the current state of the codebase. If you are interested, feel free to make a new PR for this feature, superseding this one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing weeks,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if you only keep the components that are set, .isoformat() becomes a proper inverse transformation of str → Duration
assert pendulum.parse("P1W").isoformat() == "P1W"
need same change below obviously
Pull Request Check List
This adds a simple method to represent Durations in ISO format. Given that pendulum can parse ISO 8601 strings into Durations, it makes sense that we'd be able to also serialize the other way.